Skip to content

feat: add storybook to the framework list#1649

Merged
danielroe merged 1 commit intonpmx-dev:mainfrom
JReinhold:storybook-framework-list
Feb 25, 2026
Merged

feat: add storybook to the framework list#1649
danielroe merged 1 commit intonpmx-dev:mainfrom
JReinhold:storybook-framework-list

Conversation

@JReinhold
Copy link
Collaborator

🔗 Linked issue

🧭 Context

📚 Description

Adds Storybook to the list of showcased frameworks.

@vercel
Copy link

vercel bot commented Feb 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 25, 2026 9:11pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 25, 2026 9:11pm
npmx-lunaria Ignored Ignored Feb 25, 2026 9:11pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 25, 2026

📝 Walkthrough

Walkthrough

This pull request introduces Storybook as a listed framework within the application by adding a new entry to the SHOWCASED_FRAMEWORKS array in app/utils/frameworks.ts. The entry includes the framework name 'storybook', its corresponding package identifier 'storybook', and an associated oklch colour value. This change expands the public list of frameworks, allowing getFrameworkColor and isListedFramework to recognise Storybook as a supported framework.

Possibly related PRs

Suggested reviewers

  • danielroe
  • ElianCodes
  • serhalp
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly states the change: 'Adds Storybook to the list of showcased frameworks,' which directly relates to the changeset that adds a Storybook entry to the SHOWCASED_FRAMEWORKS array.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
app/utils/frameworks.ts (1)

69-73: Colour format inconsistency: percentage vs decimal lightness.

The new entry uses 67.88% for the lightness component, whilst the majority of existing entries use the decimal form (e.g., 0.6788). A handful of existing entries (nitro, next, tanstack start) already use the percentage form, so this is not introduced by this PR, but it is worth normalising across all entries to avoid confusion.

♻️ Decimal normalisation for the new entry (consistent with the majority)
-    color: 'oklch(67.88% 0.2222 5.18)',
+    color: 'oklch(0.6788 0.2222 5.18)',

ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 40d1964 and 16fd49d.

📒 Files selected for processing (1)
  • app/utils/frameworks.ts

@codecov
Copy link

codecov bot commented Feb 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@danielroe danielroe added this pull request to the merge queue Feb 25, 2026
Merged via the queue into npmx-dev:main with commit 699868d Feb 25, 2026
17 checks passed
@github-actions
Copy link

Thanks for your first contribution, @JReinhold! ⭐

We'd love to welcome you to the npmx community. Come and say hi on Discord! And once you've joined, visit npmx.wamellow.com to claim the contributor role.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants